Merge branch 'benzea/foreach-documentation' into 'master'

container: Document that removal from foreach is permissible

Closes #1461

See merge request GNOME/gtk!423

container: Document that removal from foreach is permissible

It is permissable to remove a widget using gtk_container_remove from the
gtk_container_foreach callback handler. Document this fact to make it
more discoverable.

Fixes #1461

Merge branch 'akitouni/gdkvulkan-doc-typo' into 'master'

vulkancontext: fix typos in documentation

See merge request GNOME/gtk!417

Merge branch 'more-nullable' into 'master'

g-i: Add some nullable annotations

See merge request GNOME/gtk!419

g-i: Add some nullable annotations

Based on grepping arg docs for NULL.
See https://gitlab.gnome.org/GNOME/pygobject/issues/261

vulkancontext: fix typos in documentation

Merge branch 'jjardon/no_G_TYPE_INSTANCE_GET_PRIVATE' into 'master'

Stop using G_TYPE_INSTANCE_GET_PRIVATE completely

See merge request GNOME/gtk!402

Merge branch 'include-ft2build.h' into 'master'

font chooser: Properly include freetype headers

Closes #1432

See merge request GNOME/gtk!413

widget: Fix size-allocate signal docs

Use g_clear_pointer to unparent widgets

widget: Don't pass a position to ->size_allocate

The values have been 0/0 for a long time now, so just drop the
GtkAllocation argument and replace it with width and height.

docs: Add GtkEventControllerLegacy

vulkancontext: Add missing error switch

appchooserbutton: Remove priv pointer

testappchooserbutton: Fix for class hierarchy change

appchooserbutton: Inherit from GtkWidget

render: Fix a position/size mixup

Fixes the foreigndrawing demo.

toolbar: Remove dead code

The size definitely changes, otherwise ->size_allocate does not get
called.

toolbar: Remove some unused defines

wayland: Adapt to settings portal api change

ReadAll now accepts an array of patterns.

Merge branch 'scroll-event-propagation' into 'master'

eventcontrollerscroll: Conditionally propagate ::scroll

Closes #45

See merge request GNOME/gtk!250

eventcontrollerscroll: Conditionally propagate ::scroll

Currently, gtk_event_controller_scroll_handle_event() always returns
TRUE if it is handled, which stops the propagation of the event. If
there’s a single GtkEventControllerScroll in the widget hierarchy, that
means that no others will run, depending on the propagation phase. In
Nautilus, this can be observed when adding a scroll controller to the
GtkScrolledWindow (ctrl-scrolling controls the zoom level) - either the
scrolling or the zooming breaks.

Fixes https://gitlab.gnome.org/GNOME/gtk/issues/45

font chooser: Properly include freetype headers

freetype headers should be included via <ft2build.h> and then
indirectly via FT_*_H macros.

Fixes issue #1432.

calendar: Remove unused struct member

scrolledwindow: Remove some useless code

This is all being done already be the add_class/remove_class calls
before.

scale: Don't set label css name to "label"

That's already the css name of all labels.

colorscale: Remove x/y arguments from snapshot_scale

We only pass 0 anyway.

Adwaita: use HIG grey for bg_color

- refresh with a light bg color, use the new color palette

Adwaita: fix osd bg color

- don't try deriving from bg_color, osd should be consistent
  for light & dark.

FIxes issue #1449

Adwaita: Propagate bg color to titlebar separator descendants

Gives the same background color to all separators descending from a
title bar than to its direct childrens.

This prevents separators which are in a titlebar but not direct children
from the widget with the titlebar style class from being almost
transparent and hence it prevent them from revealing the clear color of
the window's titlebar (black).

https://gitlab.gnome.org/GNOME/gtk/issues/1231

Merge branch 'wip/jimmac/insensitive-progress' into 'master'

Adwaita: draw insensitive progressbars

See merge request GNOME/gtk!360

Merge branch 'wip/jimmac/hig-colors-gtk4' into 'master'

Adwaita: use new HIG colors

See merge request GNOME/gtk!407

Adwaita: use new HIG colors

- blue selection color, success & destructive colors updates
- blueish tint for dark

Addresses issue #1443

testpopup: Fix GtkEventControllerMotion::motion callback signature

eventcontrollerlegact: Fix ::event docs

As discussed in !388.

icontheme: Stop exporting gtk_icon_info_new_for_file

icontheme: Stop exporting gtk_icon_theme_color_symbolic_pixbuf

icontheme: Drop remaining emblem code

icontheme: Don't reference nonexistent parameter in docs

icontheme: Use a bit field in AsyncSymbolicData

176 -> 144 bytes

icontheme: Stop strdup'ing css colors when loading

... symbolic icons.

gl renderer: Properly retrieve matrix scale

So rotating offscreen nodes works.

widget: Avoid some more ->priv accesses

range: Remove multipress_gesture member

We don't need it in the instance struct since we only set it up and then
access it in its signal callbacks.

widget: Some whitespace fixes

Popover: Minimally document the ::closed signal

This is better than nothing at all. The wording is taken from Carlos's
commit message when he added this shortly before 3.12 (but skip Since).
Skip the bit from his commit message explaining what this replaced; we
don't need to say all the less good things our convenience API replaces.

testtreelistmodel: Don't use non-standard function

as per efd3758f6a779041ff552cb4c5163446fd951368 strcasecmp() is not a C
standard thing (not that we bothered including any header for it anyway)
and so this test failed to build on Windows with Microsoft Visual C.

TreeMenu: Don't manually reinvent g_list_index()

g_list_index() "Gets the position of the element containing the given
data (starting from 0)." That is exactly what we were manually doing.

gtk/gtktoolbutton.c: Document why we need to still use G_TYPE_INSTANCE_GET_PRIVATE

This is the excellent explanation from Emmanuele at
https://gitlab.gnome.org/GNOME/gtk/merge_requests/402#note_361210:
"
Every time you instantiate a type, the instance_init() function is called for each
parent type T_p of your type T; to preserve invariants, the class pointer inside
the instance data is set to the parent type before each invocation, until you hit
your type T. This means that calling GET_CLASS() inside an instance_init() function
will give you a pointer to the class vtable for the parent type T_p while you're
iterating over parent types. What if you want to access the actual class vtable of
the type T, though? Well, you can because the actual signature of instance_init() is:

  void (* GInstanceInitFunc) (GTypeInstance *instance, gpointer g_class);

i.e. all instance_init() functions get passed the instance they are initialising
and the class vtable of the real type you're instantiating.

This is how GtkToolButton works: it "peeks ahead" at instance initialisation time,
to use the button_type class field of the actual type you're instantiating,
and calls g_object_new() with it to store the resulting object in its own private
data structure.

This whole contrived mechanism is needed to allow out-of-tree tool buttons to just
set the button type on their class init, and have their parent class create the
button they want, instead of asking all tool buttons to do this themselves and have
a virtual function called get_button() for GtkToolButton to call whenever it needs
to operate on the button instance.

Now we're coming to a close: we cannot use the G_DEFINE_TYPE macro because the
instance_init() function it creates internally will not pass the class pointer
to your custom instance_init(). Since we cannot use G_DEFINE_TYPE, we also cannot use
G_ADD_PRIVATE either.

This is the reason why, when I ported GTK 3 to the new private instance data structure
macros, I left GtkToolButton alone. I should have left a comment there, because @matthiasc
tried doing that as well, and then had to revert it in commit 1c4a7bd5. So: my bad,
sorry about that.

If we want to drop the G_TYPE_INSTANCE_GET_PRIVATE and the g_type_class_add_private() calls,
we cannot use G_DEFINE_TYPE, but what we can do is unrolling what the macros do themselves:

- add a global GtkToolButton_private_offset variable
- add a static inline gtk_tool_button_get_instance_private() that does return
(G_STRUCT_MEMBER_P (self, GtkToolButton_private_offset));
- call g_type_add_instance_private (g_define_type_id, sizeof (GtkToolButtonPrivate)) inside
gtk_tool_button_get_type() and store the result in GtkToolButton_private_offset
- replace g_type_class_add_private() inside gtk_tool_button_class_init() with
g_type_class_adjust_private_offset (klass, &GtkToolButton_private_offset)
"

gtk/gtktextview.c: Remove unused GTK_TEXT_VIEW_GET_PRIVATE() macro

Merge branch 'wip/3v1n0/recent-info-cleanup' into 'master'

gtk-autocleanups: add cleanup function for GtkRecentInfo

See merge request GNOME/gtk!409

wayland: Fix a thinko in settings portal support

When we decide to fall back because the settings portal
is not present, adhere to that decision elsewhere. And
treat the fontconfig-timestamp like the other special-cased
settings, with G_TYPE_NONE.

gtk-autocleanups: add cleanup function for GtkRecentInfo

Merge branch 'wip/settings-portal' into 'master'

settings portal

See merge request GNOME/gtk!403

wayland: Support the settings portal

Under Wayland, we are currently directly using GSettings
for desktop settings. But in a sandbox, we may not have
access to dconf, so this may fail. Use the new settings
portal instead.

Move the should_use_portal helper to gdk

We want to use it there too, in the future.
Update all callers.

Use a simpler sandbox check

No need to use the runtime dir and allocate a string.
We can just check in /.

Merge branch 'wip/fmuellner/deco-setting' into 'master'

wayland: Remove GNOME Classic-specific workaround

See merge request GNOME/gtk!400

wayland: Remove GNOME Classic-specific workaround

As GSettings now supports session-specific defaults, GNOME Classic
no longer uses a separate schema and the decoration layout is always
determined by the regular schema.

This essentially reverts commit add67b516cb6 (although the code was
moved since then).

https://gitlab.gnome.org/GNOME/gtk/merge_requests/400

GdkQuartz: Handle NULL from [NSGraphicsContext currentContext].

By returning a default surface. The situation where there's no
currentContext arises when GtkCSS is trying to determine the
layout sizes so no actual display is necessary.

Closes: #1411

Use [NSGraphicsContext CGContext] instead of graphicsPort after Yosemite.

Update GDK_OSX versions to current MacOS release.

Update POTFILES.in

Merge branch 'wip/carlosg/public-legacy-controller' into 'master'

Make legacy controller public

See merge request GNOME/gtk!388

Merge branch '1422-gtkentry-s-minimum-width-is-hardcoded-to-150px-GTK4' into 'master'

gtkentry: set minimum-width to 0 instead of 150

Closes #1422 and evince#1002

See merge request GNOME/gtk!393

Updated Slovenian translation

Added Slovenian translation

gtkentry: set minimum-width to 0 instead of 150

and use 150 as natural-width.

Currently there's no way for a GtkEntry to be less
than 150px wide (apart from using "width-chars" property),
this is too much for a default minimum-width, an app
developer may need to have a shorter GtkEntry, for example
when the UI it's been shrunk by the user (see [1]) or when
you want to match the size of another widget (which is less
than 150px) see [2] for Evince bug on using
gtk_combo_box_new_with_model_and_entry() for PDF forms where
GtkEntry of ComboBox is too wide and doesn't match the combo
list width.

Using "width-chars" property may be a workaround to obtain
a short minimum-width for the entry, but is not a proper
solution for the mentioned cases as you may not know how
short your GtkEntry will be, or the fact that using "chars"
as a width unit is not pixel accurate.

Curious note: the commit that introduced the GtkEntry
minimum-width to be 150px is from 20 years ago, see
https://bit.ly/2ySEfK4

[1] This change was already suggested by Benjamin Otte
in a blog comment https://bit.ly/2J96wRo

[2] Fixes issue evince#1002

Merge branch 'nirbheek/meson-fallback-dep-fixes' into 'master'

Misc meson fixes for using subproject dependencies

See merge request GNOME/gtk!232

Update Esperanto translation

Adwaita: prevent devel styling break selection mode

- Selection mode does not get the special devel styling.

- removed teh last-child() selector for it doesn't work anymore.
  Better style all section of the headerbar than none. Proper fix pending.

https://source.puri.sm/Librem5/libhandy/issues/57

quartz: do not cache the screen in the gdkmonitor

Instead we just cache the monitor number and get
out of it the nsscreen when it is needed. This is
a requirement since it nsscreen it is not supposed
to be cached.

Fixes: https://gitlab.gnome.org/GNOME/gtk/issues/1312

Merge branch '1397-gtknotebook-built-in-popup-menu-listing-tabs-doesn-t-use-tab-label-text-for-the-last-tab-4' into 'master'

Notebook: Ensure menu label updates with tab_label

Closes #1397

See merge request GNOME/gtk!386

Merge branch 'lrn/zorder-gtk4' into 'master'

Don't let the OS maintain relative Z-order for windows (GTK4)

See merge request GNOME/gtk!197

tests: Drop "saved positions" test in testgtk

This test solely relies on configure events, which aren't supported anymore.

tests: Drop ::event handler in testgtk

We can use ::size-allocate for it, as window positions in root coordinates
are neither supported nor crucial here.

testsuite: Fix gestures tests for the legacy controller

Let it create one with the right propagation phase, so the event propagation
chain stays the same.

tests: Use GtkEventControllerMotion in testpopup

Track motion through it, instead of a GtkWidget::event handler.

tests: Remove ::event handler in testwindowsize

We just want widget size here, not windowing configuration events.

gtkwidget: Drop obsolete code

This path is not being hit anymore, gtkmain.c does handle those events
itself.

testsuite: Fix gestures test

Was broken by the removal of the default GtkEventControllerLegacy.

gtkeventcontrollerlegacy: Make public

And move ::event into it, dropping the GtkWidget::event signal.

Notebook: Ensure menu_label updates with tab_label

This was noticed in Firefox and demonstrated using a GtkBuilder ui file.
buildable_add_child() calls set_tab_label(), but the latter did nothing
to update the menu_label corresponding to that tab with the new text.
Using Builder to populate the tab child, only tabs other than last got
the right non-default labels, and even that was mostly coincidental, as
adding the main child called update_labels() via real_insert_page(), so
it took effect when the 2nd last main child is added, updating the rest
but leaving the last with the default label, not that given in Builder.

Fix by factoring out the code from child_reordered() to a new helper
menu_item_recreate() and calling that in set_tab_label(), so that
whenever the tab_label is updated, so is its corresponding menu_label.

This fixes the reported case and presumably others that we could write.

fixes https://gitlab.gnome.org/GNOME/gtk/issues/1397

Notebook: Don't notify 2x from set_tab_label_text

It calls set_tab_label(), which already does that.

gtk-demo/main: Suppress implicit fallthru warning

Comments matched to reassure the compiler that fallthrough is
intentional are supposed to precede the case or default keywords, at
least in GCC, so the one here did not suppress the warning with GCC. We
can just the if condition and put the comment at the end to solve that.

https://developers.redhat.com/blog/2017/03/10/wimplicit-fallthrough-in-gcc-7/

.gitignore: Ignore all subdirs in the subprojects dir

meson: Always fetch pangoft2_dep from the pango subproject

This ensures that we will use pangoft2 if it is available, and not if
it is not available.

See: https://gitlab.gnome.org/GNOME/pango/merge_requests/6

meson: Use proxy-libintl subproject when needed and available

Such as on Windows with MSVC.

meson: Add a fallback for the harfbuzz dependency

This meson port is not upstream yet, so a wrap file is not included.
Upstream has expressed interest but the port hasn't been tested on all
platforms yet. Will be added when it gets upstreamed.

Link to WIP port: https://github.com/centricular/harfbuzz

emojichooser: Pass chooser to add_emoji

widget: Stop adding a legacy event controller

The only event signal left is ::event and everything needing that
connects to it directly.

gl renderer: Save some matrix multiplications

We do this for every single node, which is a little costly, especially
since the common case for the modelview matrix these days is a simple
translation. So, check whether the new modelview matrix is only a
translation matrix and if so, don't do a full matrix multiplication per
node.

spinbutton: Don't use gfloat in a code sample

Merge branch 'file-sealing' into 'master'

gdk: seal in-memory files when possible

See merge request GNOME/gtk!353

Merge branch 'gtkbuilder-tag-end-wrong-type' into 'master'

GtkBuildable: Fix the type of the user_data in GtkBuildable.custom_tag_end

See merge request GNOME/gtk!378

gtkimcontextime.c: Fix Korean input

Commit 64a489ad inadvertently introduced a regression that broke Korean
text input because the changes there resulted that only the last input
string that we have from ImmGetCompositionStringW() for each time the
commit signal is emitted is kept, and also as a result the final Korean
character that is input by hitting space is also lost as a result, as we
didn't check for whether we are done with preediting.

Fix these issues by doing the following when we receive the
WM_IME_COMPOSITION message with GCS_RESULTSTR from Windows:
-Do not emit the commit signal during WM_IME_ENDCOMPOSITION, and...
-Emit the commit signal anyways, as we did before c255ba68, however...
-We still save up the string to commit, because we need to re-compute
 the cursor position when we do ->get_preedit_string(), which needs to
 take the GCS_RESULTSTR string we get from WM_IME_COMPOSITION into
 account as well, so that we avoid getting the Pango criticals that
 occur during Chinese (and most likely Japanese) input as the cursor
 position is out-of-range.

Fixes issue #1350.

testsuite/gtk/filterlistmodel.c: Avoid GCCism

Pointer arithmetic on gpointers (void *) is a GCCism, so avoid that by
being more explicit with things.

testsuite/gtk/[flatten|sort]listmodel.c: Avoid VLA usage

Visual Studio is unlikely to support VLAs at any point, so avoid using
them and use g_newa() instead.

GtkBuildable: Fix the type of the user_data in GtkBuildable.custom_tag_end

The previous type was a pointer to a pointer, which seems to be a copy-paste
error from GtkBuildable.custom_tag_start which is an out parameter. It was
always cast in use so this is an API break, but not an ABI one.

Merge branch 'master' into 'master'

gtkstack: fix null pointer dereference

See merge request GNOME/gtk!361